Repository navigation
Tests: Cover erase personal data request flow in admin E2E - #15668
shail-mehta wants to merge 2 commits into
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
Both assertions check row visibility rather than the exact requester email link, so they can pass even if the Requester column no longer displays the address.
1 open issue
What changed in this PR
Adds admin end-to-end coverage for creating an Erase Personal Data request, supporting ticket #52895 without running the destructive erasure.
Changes:
- Creates a temporary subscriber and submits a request by username with confirmation email disabled.
- Checks the success notice and requester row before and after searching.
- Adds teardown to delete the request and temporary user.
| File | Description |
|---|---|
tests/e2e/specs/erase-personal-data.test.js |
Adds request-creation coverage with user setup, list checks, and cleanup. |
Review effort: Balanced
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. Props Bot commandsYou can trigger Props Bot actions by commenting on this PR:
|
lancewillett
left a comment
lancewillett
left a comment
There was a problem hiding this comment.
Thanks @shail-mehta. Adding a request from the form is not covered yet, and creating your own user keeps it safe in parallel.
This spec and #15688 (export) are the same test on two screens: the same setup, cleanup, steps and checks, apart from the screen, the checkbox label and the username prefix. Please combine them into one spec that loops over both screens, the way taxonomies.test.js loops over categories and tags, and close the other PR. That halves the code to maintain.
AI review · claude-opus-5-5
Trac ticket: #52895
Adds an admin e2e test for creating an Erase Personal Data request from ticket #52895.
What
erase-personal-data.phpWhy
Ticket #52895 / comment feedback: create a test user, add an erasure request, and verify they appear under Requester—without executing the destructive erase.
How
New Playwright spec:
tests/e2e/specs/erase-personal-data.test.js, following existing admin e2e patterns.Testing instructions
npm run env:startnpm run test:e2e -- tests/e2e/specs/erase-personal-data.test.jsUse of AI Tools